MaskRay added a comment.

In D148785#4348053 <https://reviews.llvm.org/D148785#4348053>, @vitalybuka 
wrote:
> Is possible to split the patch into smaller ones?

The Clang CodeGen side needs to match 
`llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp` and `compiler-rt/lib/ubsan`, so 
no, I think this is the smallest unit...

I have placed logically independent parts in other patches: D148665 
<https://reviews.llvm.org/D148665>, D148573 <https://reviews.llvm.org/D148573>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148785

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to