[SandboxVec] Clear Context's state within runOnFunction() (#124842)

`sandboxir::Context` is defined at a pass-level scope with the
`SandboxVectorizerPass` class because the function pass manager `F

[SandboxVec] Clear Context's state within runOnFunction() (#124842)

`sandboxir::Context` is defined at a pass-level scope with the
`SandboxVectorizerPass` class because the function pass manager `FPM`
object depends on it, and that is in pass-level scope to avoid
recreating the pass pipeline every single time `runOnFunction()` is
called.

This means that the Context's state lives on across function passes. The
problem is twofold:
(i) the LLVM IR to Sandbox IR map can grow very large including objects
from different functions, which is of no use to the vectorizer, as it's
a function-level pass.
(ii) this can result in stale data in the LLVM IR to Sandbox IR object
map, as other passes may delete LLVM IR objects.

To fix both issues this patch introduces a `Context::clear()` function
that clears the `LLVMValueToValueMap`.

show more ...

[SandboxIR] createFunction() should always create a function (#124665)

This patch removes the assertion that checks for an existing function.
If one exists it will remove it and create a new one. T

[SandboxIR] createFunction() should always create a function (#124665)

This patch removes the assertion that checks for an existing function.
If one exists it will remove it and create a new one. This helps remove
a crash when a function declaration object already exists and we are
about to create a SandboxIR object for the definition.

show more ...

[SandboxVectorizer][NFCI] Fix use of possibly-uninitialized scalar. (#122201)

The `EraseCallbackID` field is not always initialized in the ctor for
SeedCollector; if not, it will be used uninitializ

[SandboxVectorizer][NFCI] Fix use of possibly-uninitialized scalar. (#122201)

The `EraseCallbackID` field is not always initialized in the ctor for
SeedCollector; if not, it will be used uninitialized by its dtor. This
could potentially lead to the erasure of a random callback, leading to a
bug.

Fixed by making `CallbackID` an opaque type, which is always
default-initialized to an invalid ID.

show more ...

[SandboxIR] Fix a warning

This patch fixes:

llvm/lib/SandboxIR/Context.cpp:684:22: error: unused variable
'MaxRegisteredCallbacks' [-Werror,-Wunused-const-variable]

[SandboxIR] Add callbacks for instruction insert/remove/move ops (#112965)

[SandboxIR] Switch more Instruction::create() to InsertPosition (#111213)

Also moves CmpInst definitions from Context.cpp to Instruction.cpp

[SandboxIR][NFC] Move Function class to a separate file (#110526)

[SandboxIR][NFC] Move Instruction classes into a separate file (#110294)

[SandboxIR][NFC] Move User into a separate file (#110157)

[SandboxIR][NFC] Move Context class into a separate file (#110049)