[PATCH] D122405: [llvm] Initialize and move UseUpRegs outside of `union` MemOp struct.

Thu Mar 24 20:45:40 PDT 2022

pengfei added a comment.

In D122405#3406886 <https://reviews.llvm.org/D122405#3406886>, @xiangzhangllvm wrote:

> First thanks very much for this patch!
>
> I didn't understand why Buildbot fail on clang/test/CodeGen/ms-inline-asm.c on sanitizer-x86_64-linux-bootstrap-ubsan
>
>   This test passed in my local machine (before/without this patch).
>   $../build-all/bin/llvm-lit clang/test/CodeGen/ms-inline-asm.c
>   -- Testing: 1 tests, 1 workers --
>   PASS: Clang :: CodeGen/ms-inline-asm.c (1 of 1)
>   
>   Testing Time: 0.51s
>     Passed: 1

Not sure if you need to run asan check mannually. Here are config for `sanitizer-x86_64-linux-bootstrap-ubsan`: https://github.com/llvm/llvm-zorg/blob/main/zorg/buildbot/builders/sanitizers/buildbot_cmake.sh

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122405/new/

https://reviews.llvm.org/D122405