Issue 130553
Summary MachineSanitizerBinaryMetadata modifies original IR as a MachineFunction pass
Labels llvm:codegen
Assignees melver, dvyukov, vitalybuka
Reporter arsenm
    https://github.com/llvm/llvm-project/blob/c4808741e89df29dcd06c7be2784824c82f34e46/llvm/lib/CodeGen/SanitizerBinaryMetadata.cpp#L82

This violates the rules of codegen, where we assume the underlying IR never changes. 

For the purposes of what this pass is doing, it should not be a machine pass. It is only checking the size and alignment of stack passed arguments. The ABI is exactly computable from an IR signature, and does not require going through codegen to get this information.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to