steakhal requested changes to this revision.
steakhal added a comment.
This revision now requires changes to proceed.

Thanks for looking into this.

I could not reproduce the issue with the following test code:  
https://godbolt.org/z/hsY9PTKGz

  struct A {
    unsigned x : 3;
    unsigned : 29; 
    unsigned y;
  };
  void func1(A a);
  
  void func2() {
    A a = {0};
    func1(a); // no-warning here on trunk
  }

I'd recommend adding a test that would demonstrate that before your fix the 
test would diagnose a FP and after your fix, the FP would disappear.
Without tests, unfortunately, we cannot accept patches.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146194

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

Reply via email to